-
Notifications
You must be signed in to change notification settings - Fork 10
🎨 DSFR - Admin & Mon DomiFa - Branche principale ⭐ #3855
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
6de7b54
to
7402835
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An error occurred: ##
Issues Found
1. This PR changes 49 files, which exceeds the limit of 25 files.
Suggestion: Consider breaking this PR into smaller, more focused changes. Large PRs are harder to review effectively and may contain unrelated changes.
PR Metrics
- Total files changed: 49
- Reviewable files: 49
- Diff size: 2190 lines
- Documentation files: 0
- Largest file change: 408 lines
- Addition/Deletion ratio: 0.93
This validation helps ensure the bot focuses on PRs where automated review provides the most value. You can adjust these limits in your .revu.yml
configuration file.
7402835
to
1bd54cf
Compare
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
1bd54cf
to
bbb883c
Compare
bbb883c
to
b45904d
Compare
b45904d
to
d476827
Compare
* feat(mon-domifa): init dsfr * fix(yarn): fix yarn.lock * fix(yarn): fix yarn.lock * feat(mon-domifa): init dsfr * feat(front): footer dsfr portails admin & usagers * fix(admin-usagers): mise en conformité à la NOUVELLE maquette --------- Co-authored-by: Yassine R. <riffi.yassine@gmail.com>
* feat(usagers): integration header mon domifa * feat(usagers): fix reviews * fix(usagers): fix reviews
|
[severity]="toast.class === 'success' ? 'success' : 'error'" | ||
[ariaRoleValue]="'alert'" | ||
[hasAriaRole]="false" | ||
[closeable]="toast.dissmissable" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ca ne fonctionne pas sur la démo. Je suis d'avis de le retirer pour l'instant, on va perdre du temps à essayer de corriger le souci
link: '/plan-site', | ||
}, | ||
{ | ||
label: 'Nos partenaires', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hbalty a retirer
link: '/cgu', | ||
}, | ||
{ | ||
label: 'Polotiques de confidentialité', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hbalty typo
données, consultez notre</span | ||
> | ||
<a | ||
rel="noopener external" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
le "rel" est à retirer, on est sur un lien interne @hbalty
link: '/plan-site', | ||
}, | ||
{ | ||
label: 'Nos partenaires', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A retirer dans l'admin & mon domifa
No description provided.